Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added link to docs #486

Merged
merged 3 commits into from Oct 23, 2019
Merged

Conversation

Fl4m3Ph03n1x
Copy link
Contributor

Adding more information on where to go for the ssl_options flag.
Follow up from this forum discussion:

https://elixirforum.com/t/how-to-connect-to-postgrex-with-ssl-certificate/26255

lib/postgrex.ex Outdated
@@ -79,7 +79,8 @@ defmodule Postgrex do
* `:handshake_timeout` - Connection handshake timeout in milliseconds
(defaults to `:timeout` value);
* `:ssl` - Set to `true` if ssl should be used (default: `false`);
* `:ssl_opts` - A list of ssl options, see ssl docs;
* `:ssl_opts` - A list of ssl options, see the
[ssl docs](http://erlang.org/doc/man/ssl.html);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good call. How about linking to particular function:

* `:ssl_opts` - A list of ssl options, see `:ssl.connect/2`;

?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree that more precise documentation would be more helpful, it's just that as this is my first PR into this project I was unsure if my suggestions was too specific.

From my understanding, the connect options the ssl_opts expects is actually tls_client_option.

:ssl_opts - A list of ssl options, see the tls_client_option from the ssl docs;

Would this be accurate?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup, let's go with that 👍.

@hauleth
Copy link
Contributor

hauleth commented Oct 23, 2019

Partially this was done in the #485, but I agree, that some more information in docs would be helpful for newcomers.

👍

@wojtekmach
Copy link
Member

@hauleth oh, nice, I missed that pr. @Fl4m3Ph03n1x could use also change that type to use tls_client_option as its more specific?

@Fl4m3Ph03n1x
Copy link
Contributor Author

I did some modifications, please let me know if I missed something !

@wojtekmach wojtekmach merged commit a8b5f4a into elixir-ecto:master Oct 23, 2019
@wojtekmach
Copy link
Member

Thank you!

@Fl4m3Ph03n1x Fl4m3Ph03n1x deleted the improving_docs branch October 23, 2019 12:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants